Migrate admin app from Next.js App Router to TanStack Start#47
Draft
lorenzocorallo wants to merge 12 commits into
Draft
Migrate admin app from Next.js App Router to TanStack Start#47lorenzocorallo wants to merge 12 commits into
lorenzocorallo wants to merge 12 commits into
Conversation
- Replace Next.js app shell with TanStack Start, Vite, and Nitro - Remove legacy app router, shadcn scaffolding, and Next build config - Update Docker, scripts, and lockfile for the new runtime
- Add Tailwind CSS v4 and shadcn/ui setup - Refactor dashboard and auth screens onto the new component system - Update docs and dependencies for the new UI stack
- Fix import ordering and type imports - Add htmlFor/labels for accessibility - Replace div[role=group] with fieldset - Suppress auto-generated routeTree.gen.ts in biome check - Simplify biome check scripts
- Added explicit size-4/5/6/7 className to icon components across dashboard, login, and onboarding routes - Swapped login page logo from .png to .svg
- Added DataPageSkeleton, DetailPageSkeleton, DashboardPageSkeleton components - Added shadcn/ui Skeleton component - Set defaultPendingMs to 100ms in router config - Replaced missing h2 titles with skeleton placeholders on account page - Removed redundant h2 from DataToolbar
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…stack-start # Conflicts: # src/app/dashboard/(active)/azure/members/create-assoc-member.tsx
- Removed manual query/membersOnly/sort/pagination state from Azure/Telegram tables - Replaced with globalFilteringFeature, rowSortingFeature, rowPaginationFeature - Added page-size selector and first/last-page buttons to Pagination - Controlled search input debounced with useDeferredValue in DataToolbar - Added "Add group admin" dialog to user detail page - Removed DashboardPageSkeleton - Switched router.invalidate calls to sync mode
- Sync loader data to local state for optimistic updates - Rollback UI on save failure for edit operations - Only invalidate router on create, not edit - Disable auto page reset for table
- New ThemeToggle component with dark/light mode switching - Add dark mode CSS variables and component overrides - Remove `/dashboard/telegram/users` layout route, flatten children under `/dashboard`
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
src/app/tosrc/routes/.next,postcss,@t3-oss/env-nextjs,@trpc/next,radix-ui,cmdk,react-hook-form, and many unused shadcn components.Testing
pnpm devstarts successfully on port 3001 and hot-reloads correctly.pnpm buildcompletes without errors, producing a standalone Nitro server.pnpm startserves the production build correctly.pnpm typecheckpasses with zero type errors.pnpm checkpasses biome linting and formatting checks.